nova05: follow up changes#764
Conversation
bogdando
changed the title
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
bogdando
changed the title
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
bogdando
force-pushed
the
nova05epsilon
branch
2 times, most recently
from
ca9f9f8 to
849f177
Compare
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
bogdando
force-pushed
the
nova05epsilon
branch
2 times, most recently
from
2a291a0 to
7285cb6
Compare
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
|
Perhaps add We cannot merge this until the above provides |
fultonj
requested review from
abays and
fmount
and removed request for
karelyatin and
raukadah
bogdando
force-pushed
the
nova05epsilon
branch
4 times, most recently
from
0b386fc to
56d2519
Compare
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
bogdando
force-pushed
the
nova05epsilon
branch
4 times, most recently
from
8d18c89 to
1fb62b1
Compare
Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
The DT-level kustomization was missing service-values replacements
(telemetry, nova, glance, ovn, extraMounts), so the examples kustomize
build produced base defaults instead of DT-specific config. This caused
metricStorage.enabled=false and missing alertmanagerConfig on the
deployed OSCP.
Additionally, pvcStorageClass was hardcoded to "local-storage" but SNO
uses LVMS ("lvms-local-storage"), and alertmanager defaulted to 2
replicas which cannot schedule on a single node due to required
pod anti-affinity.
Align the kustomization layers with nova04delta:
- dt/nova/nova05epsilon/kustomization.yaml: add service-values
replacements matching nova04delta pattern
- Remove deplicated replacements for
dt/nova/nova05epsilon/control-plane/kustomization.yaml
- examples/.../control-plane/service-values.yaml: add full DT-specific
data (telemetry, nova, glance, ovn, extraMounts)
- Set pvcStorageClass to lvms-local-storage and alertmanagerConfig
replicas to 1 for SNO compatibility
- update control-plane.md
Generated-by: claude-4.6-opus-high
Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
|
This change depends on a change that failed to merge. Change openstack-k8s-operators/ci-framework#3773 is needed. |
HCI uses edpm post ceph to also patch the OSCP which looks suboptimal.
Separate this into a two steps:
* make post-ceph nodeset layer handling EDPM resources only (NodeSet,
Services, Secrets, ConfigMap) - remove OSCP layer resources
* add control-plane-post-ceph layer for OSCP with Glance RBD + Ceph
extraMounts and secret config. Instead of re-including
lib/control-plane (which required network-values duplication)
provide only required values for lib/control-plane/base.
* update README.md stages list from 4 to 6 steps, automation step
count from 9 to 10
* add note for defferred Ceph config for control-plane.md
* add control-plane post-ceph doc for the dedicated Ceph
control-plane update stage
* remove control-plane update references for
dataplane-post-ceph.md, add prerequisite for
control-plane-post-ceph
* add control-plane-post-ceph stage between pre-ceph deployment and
post-ceph nodeset ini automation vars
Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
|
Finally works with my local testing |
|
downstream testing completed for this DT and dependency patches, it is good to go |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: bogdando, evallesp The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
New changes are detected. LGTM label has been removed. |
|
Build failed (check pipeline). Post ✔️ noop SUCCESS in 0s |
Signed-off-by: Bohdan Dobrelia <bdobreli@redhat.com>
|
@abays the deduplication patch worked for my testing |
4 participants
Add cross-site routes/GWs
For all nova05epsilon's NADs, NetConfig subnets, host-level NNCP routes add missing cross-site DCN routes,
and gateways.
Previsourely those were tested in ci-framework jobs overlays but the base DT example was missing those
CHANGEME templates.
Add SNO Swift fix hook for Ceph RGW
Ext net for tests accessing VMs via FIP
Add external NAD definition with VLAN to DT nncp/values.yaml.
Update scenario-vars.yaml with external NAD for Tempest access to FIP.
Add automation mocks for CI
Fix telemetry spec and kustomize chain for SNO
The DT-level kustomization was missing service-values replacements
(telemetry, nova, glance, ovn, extraMounts), so the examples kustomize
build produced base defaults instead of DT-specific config. This caused
metricStorage.enabled=false and missing alertmanagerConfig on the
deployed OSCP.
Additionally, pvcStorageClass was hardcoded to "local-storage" but SNO
uses LVMS ("lvms-local-storage"), and alertmanager defaulted to 2
replicas which cannot schedule on a single node due to required
pod anti-affinity.
Align the kustomization layers with nova04delta:
replacements matching nova04delta pattern
dt/nova/nova05epsilon/control-plane/kustomization.yaml
data (telemetry, nova, glance, ovn, extraMounts)
replicas to 1 for SNO compatibility
Separate pre and post ceph OSCP configs
HCI uses edpm post ceph to also patch the OSCP which looks suboptimal.
Separate this into a two steps:
Services, Secrets, ConfigMap) - remove OSCP layer resources
extraMounts and secret config. Instead of re-including lib/control-plane
(which required network-values duplication)
provide only required values for lib/control-plane/base.
count from 9 to 10
control-plane update stage
dataplane-post-ceph.md, add prerequisite for
control-plane-post-ceph
post-ceph nodeset ini automation vars
Depends-on: openstack-k8s-operators/ci-framework#3773
Related: OSPRH-30706